Skip to content

Move PR assistant directives into a supported Copilot instructions file - #2372

Merged
Erwan MATHIEU (wawanbreton) merged 2 commits into
CURA-13166_improve-code-review-processfrom
copilot/fix-review-comment-instructions
Sep 11, 2026
Merged

Move PR assistant directives into a supported Copilot instructions file#2372
Erwan MATHIEU (wawanbreton) merged 2 commits into
CURA-13166_improve-code-review-processfrom
copilot/fix-review-comment-instructions

Conversation

Copilot AI commented Sep 11, 2026

Copy link
Copy Markdown

The repository-wide Copilot instructions referenced a PR assistant file in an unsupported location, so those review directives were not being applied. This updates the instruction placement so the PR-review customization is loaded by Copilot as intended.

  • Instruction loading

    • Replaced the unsupported link-only contents of .github/copilot-instructions.md with the actual PR assistant directives.
    • Preserved the existing review guidance while moving it into a location Copilot recognizes.
  • Repository cleanup

    • Removed .github/copilot-instructions/pr-assistant.instructions.md, which was not a supported instruction source.
# .github/copilot-instructions.md
# Role: Pull Request Assistant

You are the Pull Request Assistant...
* Do not publish a pull request overview
* Generated comments should be as concise as possible
...

Co-authored-by: wawanbreton <601114+wawanbreton@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix code as per review comment suggestions Move PR assistant directives into a supported Copilot instructions file Sep 11, 2026
@wawanbreton
Erwan MATHIEU (wawanbreton) marked this pull request as ready for review September 11, 2026 13:11
@wawanbreton
Erwan MATHIEU (wawanbreton) merged commit bfb726a into main Sep 11, 2026
1 check passed
@wawanbreton
Erwan MATHIEU (wawanbreton) deleted the copilot/fix-review-comment-instructions branch September 11, 2026 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants